cssnode: Catch case where a style didn't change

Catch the case where a CSS style did not change and don't emit the
style-changed signal in that case.

This saves not only the emission of the signal, but also doesn't cause
invalidation in child nodes, which would previously get a PARENT_STYLE

cssnode: Change style-changed signal

Instead of having old and new style, now have a GtkCssStyleChange opaque
object that will compute the changes you are interested in for you.

This simplifies change signal handlers quite a bit and avoids lots of
repeated computation in every signal handler.

inspector: Hide request mode for non-widgets

Only widgets have a request mode.

Adwaita: menu color change on the dark variant only

file chooser: Also apply the settings

I forgot that the settings object we're dealing with here
is in delay mode, so settings won't apply immediately.

file chooser: Store size more frequently

We were only storing the dialog size on unmap, but resetting to the
stored default value more often, e.g. on focus-out. This was causing
the dialog to 'jump back' to its remembered size after the user
manually resized it, leading to frustration and bug reports.

Instead, save the dialog size on every ::size-allocate of the toplevel.
To avoid needlessly spamming dconf, only write the new value if it
changed.

Add a drag-end callback

This will be helpful in figuring out life-cycle issues
with icon widgets.

Add a dnd testcase

This has tests for drag images, widgets, hotspots.

wayland: Implement DND icon hotspot API

In Wayland, the hotspot of a DND icon is set using the buffer offset in
wl_buffer.attach. To implement this, add a private API to cause the
next wl_surface.attach to offset the new buffer with a given offset.
Setting a DND icon hotspot sets this offset while also queuing a redraw
of the window to trigger the wl_surface.attach.

https://bugzilla.gnome.org/show_bug.cgi?id=759168

placesviewrow: remove unneeded functions

These function declarations are not used nor implemented
anywhere, so remove them.

gtkstack: remove needless queue_resize

In gtk_stack_set_transition_position we should only need to
queue_resize if we are non-homogeneous, which is already done
earlier in the function.

placesview: implement available space

GtkPlacesView is a widget to display locations
in the computer, such as root ("/") and volumes,
separating the persistent devices from removable
ones.

From the latest mockups[1], GtkPlacesView would
display the available space of local drives like
partitions. This, however, is not implemented in
the current codebase.

Fix that by implementing the measurement of disk
space, and adding a new property GtkPlacesView::show-disk-usage
which controls the visibility of measured disks.

[1] https://raw.githubusercontent.com/gnome-design-team/gnome-mockups/master/nautilus/nautilus-next/other-locations.png

https://bugzilla.gnome.org/show_bug.cgi?id=759225

statusicon: plug memory leak

Adwaita: make menu bg color the same as popovers

fixes https://bugzilla.gnome.org/show_bug.cgi?id=759260

Adwaita: another try at color tweakings...

Adwaita: some more color tweaks and sass simplification

Adwaita: fix popover styling

...in the process simplified the touch-selection styling, check
and radios not fixed there since I'm going to add proper osd assets
for those (istead of forcing the dark variant assets there as before).

iconhelper_ Don't use deprecated functions

Use gtk_widget_get_direction() instead of
gtk_style_context_get_direction().

iconhelper: Remove the context argument

Instead, use the widget's context.

iconhelper: Get rid of _gtk_icon_helper_set_window()

Instead use the window of the owner widget.

iconhelper: Require a widget as construction argument

Note that the caller needs to ensure the widget lives as long as the
iconhelper as the iconhelper will not ref the widget.

statusicon: Port from iconhelper to imagedefinition

cellrendererpixbuf: Create iconhelper on-demand

Instead of storing one in the priv structure, only store an image
definition there.

This will allow future refactorings of the icon helper.

iconhelper: Pass only the CssStyle when loading pixbufs

iconhelper: Remove unused GtkStyleContext parameters

iconhelper: Pass only the CssStyle when loading iconsets

iconfactory: Move private functions into private header

iconhelper: Pass only the CssStyle when loading icons

This is the first step in replacing StyleContext usage with CssNode
usage.

toolitem: Set a css name

Don't use gtk_text_iter_begins_tag() (deprecated)

Use gtk_text_iter_starts_tag() instead.

https://bugzilla.gnome.org/show_bug.cgi?id=759092

textiter: add starts_tag() and deprecate begins_tag()

The name gtk_text_*_begins_* was used only for begins_tag(). All other
similar functions use "starts": starts_line(), starts_word(), etc.

So for consistency, add gtk_text_iter_starts_tag() and deprecate
gtk_text_iter_begins_tag().

Also change (allow-none) to (nullable), to use the new annotation.

https://bugzilla.gnome.org/show_bug.cgi?id=759092

Revert "wayland: Implement DND icon hotspot API"

This reverts commit 3ab9d96623bc30678cbffc3bf9ec5133e714e87c.

Pushed prematurely.

Remove GtkShortcutsGesture from docs

Commit c83ab24 that dropped GtkShortcutsGesture left in one last reference in
docs; this drops it as well.

gtk-demo: Add an example for direction-dependent shortcuts

wayland: Implement DND icon hotspot API

In Wayland, the hotspot of a DND icon is set using the buffer offset in
wl_buffer.attach. To implement this, add a private API to cause the
next wl_surface.attach to offset the new buffer with a given offset.
Setting a DND icon hotspot sets this offset while also queuing a redraw
of the window to trigger the wl_surface.attach.

https://bugzilla.gnome.org/show_bug.cgi?id=759168

Update Arabic translation

headerbar: Don't leak internal widgets

Make sure to clean up internal widgets in destroy, as these aren't
cleaned up when GtkContainer walks the childrens list in
gtk_container_destroy().

This also fixes a gedit crash as reported in
https://bugzilla.redhat.com/show_bug.cgi?id=1288669

https://bugzilla.gnome.org/show_bug.cgi?id=759132

Trivial code restructuring

dnd: pass hotspot to GDK

This uses the new api introduced in the previous commits.

wayland: Add a dummy implementation of gdk_drag_context_set_hotspot

This just records the hotspot coordinates. Still to do:
apply the hotspot when updating the drag window.

x11: Implement gdk_drag_context_set_hotspot

This makes the drag behavior under X11 match what we had previously.

gdk: Add api to set drag window hotspot

With GdkDragContext now being in charge of placing the window,
it needs to know about the hotspot to place under the cursor.

dnd: Remove an unused struct field

We only have one screen nowadays, so the fallback_icon is
never used.

GtkNotebook: Use gdouble in coordinate calculations

Otherwise rounding errors fool the "tab under coordinates" checks on
crossing events, which will be triggered close enough to the window
rectangle if the pointer moves slowly enough.

With this, the tab_prelight() function correctly figures out we've
moved the pointer outside the tab area when called in
gtk_notebook_leave_notify().

https://bugzilla.gnome.org/show_bug.cgi?id=759091

Fix method to update menu sensitivity of combo box

After removal of the selectable header and separator from the combo box,
the method to update the menu sensitivity must be changed as it assumes
at least two items within each sub menu and contains special handling
for the separator. Removing this fixes bug #759079.

Avoid ugly seams on half-tiled terminals

Since we're no longer doing geometry widgets, don't send
base size and increments to the window manager anymore either.
This avoids an ugly 2 pixel gap to the right and bottom of half-tiled
terminals under gnome-shell.

accel cellrenderer: Fix rendering

We were getting the "New Accelerator" text mixed with the
content of the underlying cell, since plain labels don't
have a background. Go back to putting the label in selected
state, and fix the theme to render that white-on-blue. This
was lost when we switched to using a selection sub-node.

Correct annotations for gtk_cell_layout_get_area

The function can return NULL.

https://bugzilla.gnome.org/show_bug.cgi?id=759081

inspector: Redo the CSS node page

Showing two lists in a paned was a bit awkward, and space was
getting too limited. Go back to showing just the node list at
first, and make the CSS properties available via a stack. At
the same time, add a right-click context menu to the node list
to make the name and class editing more easily available.

inspector: Avoid a crash

gkt_cell_layout_get_area may return NULL, handle it.

css node: Some more docs

places view: Use names for cursors

Use the standard name for the busy+interactive cursor.

entry: Always use cursor names

The names are the preferred API for cursors, so use it.

Add a test for required cursors

We use these cursors in GTK+, so check for their existence.

Be forgiving if cursors are missing

No need to crash here. Missing cursors are ugly, but we
shouldn't crash.

Add more examples to foreigndrawing

Show how to draw checks and radios.

Updated Slovak translation

entry completion: Reduce the timeout

Pop up the completion window after 100ms. The previous timeout
of 300ms was making completions feel slow.

https://bugzilla.gnome.org/show_bug.cgi?id=758929

widget-factory: Add an entry completion

This was missing so far.

shortcuts: Another attempt to fix up forall

gtk_window_set_titlebar does not take ownership of the headerbar,
so we need to explicitly destroy it.

gtk-demo: Plug a memory leak

Updated POTFILES.in

Remove GtkShortcutsGesture

The gesture functionality was taken over by GtkShortcutsShortcut,
so this widget is no longer needed, and it never was in a stable
release, so lets get rid of it.

Adwaita: simpler active switch background

...removing the progressbar_fill mixin in the process.

Adwaita: first stab at some visual updates

Updated Polish translation

Updated German translation

Updated Portuguese translation

Updated Portuguese translation

3.19.4

Fix make check

GtkFontChooserWidget is using a GThemedIcon in its template,
so we need to ensure that the type is registered before
loading it. This was causing the defaultvalue test to fail.

adwaita: fix property spelling mistake

https://bugzilla.gnome.org/show_bug.cgi?id=759048

GtkEntry: Recompute in text-inserted/text-deleted

The text-inserted/text-deleted handlers rely on the cursor positions
being unadjusted after the text change, so we can't do it in
buffer_notify_text.

testsuite: Fix notify tests

GtkStatusIcon tests don't work well under xwayland either, so just
skip them unconditionally.
GtkEntry now fails because the update of the im-module is no longer
deferred to an idle, and (gtk-im-simple) is not a valid module
name, so skip this property.

shortcut label: Tweak modifier key rendering

The subscript was affecting the vertical alignment too much,
so tweak the rendering of the L/R markers to avoid that. Also,
mark these as translatable.

Add a css name to GtkShortcutsShortcut

This will be useful for tweaking the gesture image details,
among other things.

Adwaita: added back progressbar.osd style

Adwaita: fix scales with marks on the dark variant

Adwaita: slight color tweaks

reftests: Add test for missing-image handling

Add a test that ensures that missing images get the icon-effect properly
applied.

iconhelper: Fold function into callers

The function takes so many arguments and is so short that it's not worth
keeping.

iconhelper: Apply icon-effect directly

Instead of creating an icon source, making sure no state is set and
therefore the icon-effect will be applied and then rendering that icon
source, just call the icon-effect apply function.

Also, the new way isn't deprecated.

css: Leftover renaming gtk-image-effect => icon-effect

The previous renaming commit was incomplete, so here we go again.

render: Split out icon-effect apply function

iconhelper: Return surface from icon-effect function

All the callers where converting to a surface anyway.

iconhelper: Refactor missing icon handling

In particular, stop using deprecated code for loading the missing image
icon.

iconhelper: Fold function into only caller

iconfactory: Drop caching infrastructure

- icon sets are essentially unused
- the icon theme does caching
- this code complicates the restructuring I'm about to do

=> Delete it!

mir: remove deprecated function call

shortcuts: Extend the accelerator syntax more

Cover cases like left+right control, and render them nicely.
The gtk3-demo builder shortcuts example shows the new
possibilities.

wayland: fix testtooltips

On Wayland, for tooltips to work as expected, the type hint must be set
to tooltips, otherwise the popup window won't be translated as a
subsurface.

Fix the test do work as expected under Wayland.

Bugzilla: https://bugzilla.gnome.org/show_bug.cgi?id=759018

gdkoffscreenwindow: use embedder to derive surfaces & scale factor

In Gdk, a GdkOffscreenWindow parent has to be the root window. This is
problematic on Wayland because the root window doesn't necessary have the
right information with regard to scale factor.

This patch proposes to rely on the embedder, if available, to derive
surfaces as well as getting the scale factor.

https://bugzilla.gnome.org/show_bug.cgi?id=758936

GtkEntry: nullable annotations

GtkEntry: Return proper constants from signal handlers

GtkTreeModel: Remove erroneous transfer annotation

GtkEntry: Remove recompute idle

Just do it directly

GtkEntry: Remove unused fields

Adwaita: some OSD rationalization

taking care of https://bugzilla.gnome.org/show_bug.cgi?id=732528 in
the process, any toolbar.osd with a positional style class attached
gets squared corners now.

Adwaita: scale with marks styling

I'm really sorry for the unlucky ones who would have to read that
code...

Adwaita: added back scale marks style